feat(frontend): add stream templates with local persistence and polis…#153
Conversation
…h stream builder UX
ogazboiz
left a comment
There was a problem hiding this comment.
hey, thanks for implementing the stream details screen! the [id]/page.tsx dynamic route is the right approach for Next.js.
a couple of things to fix before we can merge: there are merge conflicts with main (git fetch origin && git rebase origin/main). also, there's a file called frontend/app/streams/n in the diff — looks like an accidental commit, could you remove it? once those two things are sorted this looks ready!
ogazboiz
left a comment
There was a problem hiding this comment.
hey, thanks for the stream templates feature — local persistence is a nice touch!
just a couple things to sort out before this can be merged: the PR is currently conflicting with main so you'll need to do git fetch origin && git rebase origin/main and force push. also, i noticed a pnpm-lock.yaml file in the diff — the project uses npm so could you delete that and run npm install to update the package-lock.json instead? otherwise it'll cause confusion for other contributors down the line
|
Okay i will Address the the conflict |
cf8d4eb to
f3454db
Compare
|
@ogazboiz conflict resolved |
|
@ogazboiz conflict resolved |
looks great, merging this in! thanks for the contribution |
|
You are welcome |
Summary
This PR adds reusable stream templates to the Outgoing flow and upgrades the Create Stream UX.
What’s Included
localStorage./approute entry to render the wallet/dashboard flow.info,success,error)Acceptance Criteria Mapping
Files Changed
frontend/components/dashboard/dashboard-view.tsxfrontend/app/globals.cssfrontend/app/app/page.tsxHow to Test
cd frontendnpm installnpm run devhttp://localhost:3000/app.Notes
closed: Frontend: Saved Stream Templates #138